Skip to content

Comments

[telegram] Add support for disabling long polling#19592

Merged
lolodomo merged 3 commits intoopenhab:mainfrom
lsiepel:telegram-longpolling
Nov 3, 2025
Merged

[telegram] Add support for disabling long polling#19592
lolodomo merged 3 commits intoopenhab:mainfrom
lsiepel:telegram-longpolling

Conversation

@lsiepel
Copy link
Contributor

@lsiepel lsiepel commented Oct 30, 2025

Fixes #17648

Signed-off-by: Leo Siepel <leosiepel@gmail.com>
@lsiepel lsiepel requested a review from Copilot October 30, 2025 21:52
@lsiepel lsiepel added the enhancement An enhancement or new feature for an existing add-on label Oct 30, 2025
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds the ability to disable Telegram bot long polling by allowing longPollingTime to be set to 0.

  • Conditional listener registration based on longPollingTime value
  • Documentation update to clarify that setting the value to 0 disables polling

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
TelegramHandler.java Added conditional check to only register updates listener when longPollingTime > 0
README.md Updated documentation to indicate that longPollingTime can be set to 0 to disable polling

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you

@lolodomo lolodomo merged commit e823fa3 into openhab:main Nov 3, 2025
2 checks passed
@lolodomo lolodomo added this to the 5.1 milestone Nov 3, 2025
@lsiepel lsiepel deleted the telegram-longpolling branch November 3, 2025 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement An enhancement or new feature for an existing add-on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[telegram] Possibility to run telegram addon without long polling, send action-only. For traffic limited applications.

2 participants